Skip to content

[9.4] Fix testCancelOnExpiry (#146015)#146026

Open
dnhatn wants to merge 1 commit intoelastic:9.4from
dnhatn:backport/9.4/pr-146015
Open

[9.4] Fix testCancelOnExpiry (#146015)#146026
dnhatn wants to merge 1 commit intoelastic:9.4from
dnhatn:backport/9.4/pr-146015

Conversation

@dnhatn
Copy link
Copy Markdown
Member

@dnhatn dnhatn commented Apr 10, 2026

Backports the following commits to 9.4:

The test failed because it used a small keep_alive on submission,
leading to cases where data-node drivers were slow to start and the
query was cancelled before any driver had started. This PR modifies the
test to check for cancellation in two steps: submit with a large
keep_alive and wait until at least one data-node driver has started,
then adjust the keep_alive to a much smaller interval and expect
cancellation to happen.

Closes elastic#145502
@dnhatn dnhatn added :Analytics/ES|QL AKA ESQL >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.4.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants